Skip to content

Fix bug in flaky test script#16434

Merged
antoniovicente merged 6 commits intoenvoyproxy:mainfrom
davinci26:fixFlakyTestScript
May 17, 2021
Merged

Fix bug in flaky test script#16434
antoniovicente merged 6 commits intoenvoyproxy:mainfrom
davinci26:fixFlakyTestScript

Conversation

@davinci26
Copy link
Copy Markdown
Member

@davinci26 davinci26 commented May 11, 2021

Signed-off-by: Sotiris Nanopoulos sonanopo@microsoft.com

Commit Message:

Fixes #15800

There are some errors such as Error: No such container: rbe-container-453b391c-e65b-4035-bfa6-fee0cdb0cd00 where the split fails. Harden the accesses with length checks.

Additional Description:
Risk Level: Low
Testing: Manual
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A

Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
@davinci26
Copy link
Copy Markdown
Member Author

Verifying on the CI and will assign a reviewer

Sotiris Nanopoulos added 2 commits May 11, 2021 11:01
Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
Sotiris Nanopoulos added 3 commits May 12, 2021 09:52
Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
@antoniovicente antoniovicente self-assigned this May 14, 2021
Copy link
Copy Markdown
Contributor

@antoniovicente antoniovicente left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix.

@davinci26
Copy link
Copy Markdown
Member Author

/retest

@repokitteh-read-only
Copy link
Copy Markdown

Retrying Azure Pipelines:
Retried failed jobs in: envoy-presubmit

🐱

Caused by: a #16434 (comment) was created by @davinci26.

see: more, trace.

@antoniovicente antoniovicente merged commit 69effc2 into envoyproxy:main May 17, 2021
ntgsx92 pushed a commit to ntgsx92/envoy that referenced this pull request May 18, 2021
Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
Signed-off-by: Sixiang Gu <sgu@twitter.com>
leyao-daily pushed a commit to leyao-daily/envoy that referenced this pull request Sep 30, 2021
Signed-off-by: Sotiris Nanopoulos <sonanopo@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Flaky test is flaky

2 participants